-
Notifications
You must be signed in to change notification settings - Fork 302
#4 Demonstrate the usage of MapStruct for bean mappings #249
base: develop
Are you sure you want to change the base?
Conversation
Can one of the admins verify this patch? |
Why do you have to extends |
Did you fork a state that was not working? The compile error in travis looks like it is not related to map-struct. I think this is a compiler error I caused some time ago (by not following our guideline to use PRs) that has been fixed right after. You should merge the original oasp4j repo back into your PR clone repo. |
Right - somewhere along traversing the type hierarchy, the type bounds were lost - I still need to create an example that is complex enough to reproduce the issue for the MapStruct issue-tracker (my first attempts with just an abstract super class that binds the type-variable in the generic interface wasn't enough, as it worked in that case). But I'm sure it's something that can be fixed. 😉
Locally, it worked and I was able to start the application... I've rebased my commit to the latest head now. |
I think our test-coverage is still poor, but your diff does not contain a solution for the modification counter (optimistic locking) that we solved for dozer and orika. |
you should use an established libarary for that instead of solving this by hand. Resolving type variables and determining the actual type of a getter is extremely tricky since java 1.5. Most solutions out there including hibernate and spring do it wrong and are buggy. You might create a fix for this particular case but then might still contain bugs elsewhere. |
Can one of the admins verify this patch? |
1 similar comment
Can one of the admins verify this patch? |
|
The component
tablemanagement
now uses a mapper generated by MapStruct.In the Eclipse m2e-Settings, you might need to activate the automatic configuration of the annotation-processing (Window->Preferences->Maven->Annotation Processing -> Automatically configure ...).